Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jenkins release job #594

Merged
merged 39 commits into from
Jun 18, 2018
Merged

Conversation

herlo
Copy link
Contributor

@herlo herlo commented May 21, 2018

Add cd-linchpin-release config.xml
Update JenkinsfileRelease job
Make Dockerfile match what is in ci-pipeline for jenkins slave

@herlo herlo added the cd label May 21, 2018
@herlo herlo added this to the v1.5.4 milestone May 21, 2018
@herlo herlo self-assigned this May 21, 2018
@herlo herlo requested a review from p3ck May 21, 2018 21:13
@adl-bot
Copy link

adl-bot commented May 22, 2018

@adl-bot
Copy link

adl-bot commented May 22, 2018

@herlo herlo modified the milestones: v1.5.4, v1.6.0 May 22, 2018
@herlo
Copy link
Contributor Author

herlo commented May 22, 2018

[test]

@adl-bot
Copy link

adl-bot commented May 22, 2018

@herlo herlo removed this from the v1.6.0 milestone Jun 5, 2018
@herlo herlo force-pushed the jenkins_release_job branch from 7889e1c to ce5b949 Compare June 5, 2018 16:49
@adl-bot
Copy link

adl-bot commented Jun 5, 2018

Johan Belin and others added 12 commits June 14, 2018 11:00
Minimal set of changes to enable support for remote libvirt hypervisors.
```
TASK [ovirt : Provisioning resource definitions of current group]
***********************************************
task path:
/home/user/venv/lib/python2.7/site-packages/linchpin/provision/roles/ovirt/tasks/provision_resource_gr
yml:33
fatal: [localhost]: FAILED! => {
    "msg": "template error while templating string: unexpected ')'.
    String: {{ res_grp['resource_definitions']) }
    }
```
@herlo
Copy link
Contributor Author

herlo commented Jun 14, 2018

Please note, the JenkinsfileRelease is using branches in my repository. When this is reviewed, note the comments as they will be updated before merging.

@adl-bot
Copy link

adl-bot commented Jun 14, 2018

@adl-bot
Copy link

adl-bot commented Jun 14, 2018

@herlo
Copy link
Contributor Author

herlo commented Jun 15, 2018

[test]

@adl-bot
Copy link

adl-bot commented Jun 15, 2018

@adl-bot
Copy link

adl-bot commented Jun 15, 2018

Since there will likely be further testing, the testing values stayed,
but commented out. That way, it will be simple to run more testing in
the future.
@adl-bot
Copy link

adl-bot commented Jun 15, 2018

Copy link
Contributor

@p3ck p3ck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm - just check on the openstack image before merge

@@ -16,7 +16,7 @@ resource_groups:
- name: "{{ distro | default('') }}frontend"
role: os_server
flavor: m1.small
image: Fedora-Cloud-Base-26-compose-latest
image: CentOS-7-x86_64-GenericCloud-1612
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this need to be switched back? I know sk had to change this because the CentOS image was missing? Or do I have it backwards?

@herlo
Copy link
Contributor Author

herlo commented Jun 15, 2018 via email

@herlo
Copy link
Contributor Author

herlo commented Jun 18, 2018

[merge]

@adl-bot adl-bot merged commit 535ffd5 into CentOS-PaaS-SIG:develop Jun 18, 2018
@herlo
Copy link
Contributor Author

herlo commented Jul 27, 2018 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants